-
Notifications
You must be signed in to change notification settings - Fork 33
zephyrSerial: Fix misdetection arduino_serial1 #80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
DT_NODE_EXISTS value is true even if the status is disabled, so it caused misdetection. Use DT_NODE_HAS_STATUS to detect the status of the arduino_serial1 node is ok.
oops! @soburi you seemed to have missed the signed-off This doesn't inspire confidence in the checkpatch checks that we have :( |
We definitely need to run a CI with chekpatch:
|
@beriberikix can you take a look? |
Do you want to implement |
@beriberikix yes it would be a seperate PR. |
I couldn't push to |
@soburi , sorry I don't see the signed off by in |
I failed to push. |
Okay no worries, Ill drop this commit from next and then ask for the new PR to be made with these changes. |
@DhruvaG2000
|
@soburi one can't/shouldn't directly push to any of this repo's branches unless absolutely necessary. |
There was an issue that caused build failures for
nrf52840dk_nrf52840
which is what this PR aimed to fix.With this merged into main I think all variants build successfully and we should be good to tag it as the next release.